Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sw: Add sample Monte Carlo application #51

Merged
merged 1 commit into from
Sep 27, 2023
Merged

sw: Add sample Monte Carlo application #51

merged 1 commit into from
Sep 27, 2023

Conversation

colluca
Copy link
Collaborator

@colluca colluca commented Sep 26, 2023

When merging this we should add the following lines to the end of the commit message:

Co-authored-by: Fabio Cappellini <[email protected]>
Co-authored-by: Hakim Filali <[email protected]>

@colluca colluca marked this pull request as ready for review September 27, 2023 07:28
Copy link
Contributor

@fischeti fischeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Is the auto-merging still enabled in the repository? Because in that case it will be immediately merged when I approve this PR, and we cannot change the commit message to include the co-authors

@paulsc96 paulsc96 merged commit 8ae6d3b into main Sep 27, 2023
25 checks passed
@paulsc96 paulsc96 deleted the montecarlo branch September 27, 2023 14:05
JosseVanDelm pushed a commit to JosseVanDelm/snitch_cluster that referenced this pull request Dec 11, 2023
…-platform#51)

* docs: Update schema to make snax_acc strings

* hw: Move accelerators out of snitch_cluster.sv

* hw: Process accelerators outside of snitch_cluster.sv

* cfg: Update snax-mac.hjson config

* hw: Take out unnecessary parameters

* tpl: Fix python backend in mako tpl

* cfg: Make snax-mac into string

* cfg: Update snax-gemm.hjson to use string

* docs: Update schema details

* hw: Take out unnecessary comments

* hw: Renamed snax_mac to snax_mac_wrapper

* hw: Change module name to snax_mac_wrapper

* bender: Change to use snax_mac_wrapper.sv

* hw: Put comments on template

* cfg: Add _wrapper affix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants